Skip to content

Conversation

wilhelsr
Copy link
Member

Purpose

Improve PC part cards in list and two-column views by adapting their layout to the card’s available width.

Changes

  • Use container queries to adjust image, text, price, and button placement.
  • Make cards open details through clicking or pressing Enter/Space.
  • Remove the separate “Show details” button.
  • Keep build actions from also opening details.
  • Update card tests for the new interaction.
  • Center mobile notifications and animate them from the top.
  • Respect reduced-motion preferences for animations.

Related issues

Closes #129

Verification

  • npm test — all 106 tests passed after updating the card interactions.
  • Tests cover reopening details, retrying failures, repeated clicks, request cancellation, and separate build actions.
  • npm run build — previously blocked by the skeleton’s removed detailsAction prop; rerun after the latest fixes.
  • Manual testing across screen sizes remains to be completed.

Screenshots

image image image image

Checklist

  • I have reviewed my own changes.
  • The changes meet the linked issue's acceptance criteria, where applicable.
  • I have verified the changes and recorded the results above.
  • I have added or updated tests where needed.
  • I have included screenshots for visual changes, or marked them not applicable.

@wilhelsr wilhelsr requested a review from a team as a code owner November 14, 2026 11:58
Copy link
Member

@krgjert krgjert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work, the card are now more clean

@krgjert krgjert merged commit 25607e3 into dev Nov 14, 2026
@krgjert krgjert deleted the 129-fix-styling-of-the-pcpartcard-in-all-sizes branch November 14, 2026 15:19
Sign in to join this conversation on GitHub.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants